Skip to content

fix: correct release update times and asset markers - #280

Merged
AmintaCCCP merged 1 commit into
mainfrom
fix/release-asset-update-indicators
Aug 19, 2026
Merged

fix: correct release update times and asset markers#280
AmintaCCCP merged 1 commit into
mainfrom
fix/release-asset-update-indicators

Conversation

@AmintaCCCP

@AmintaCCCP AmintaCCCP commented Aug 19, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Owner

Summary

  • aggregate repository-card release time across all visible releases and asset update times
  • track newly added and changed asset IDs during refresh
  • show the asset-level 资产已更新 / Asset updated marker in both release views

Validation

  • npm run lint (passes with one pre-existing warning in VectorSearchSettings.tsx)
  • npm run test:run (299 tests passed)
  • npm run build (passes)

Fixes #263

Summary by CodeRabbit

  • New Features

    • Release timelines now account for asset updates when determining the latest activity.
    • Updated assets are clearly marked in both direct-download and RPC download entries.
    • Repository headers display the effective update time and indicate when assets changed.

  • Bug Fixes

    • Added accurate tracking for newly added and modified release assets.
    • Prevented removed or unchanged assets from being incorrectly labeled as updated.
    • Improved handling of invalid or missing publication dates.

@coderabbitai

coderabbitai Bot commented Aug 19, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a1d9ef53-3c61-4a13-beea-528bd627ec97

📥 Commits

Reviewing files that changed from the base of the PR and between 7ce0b3c and e8a1fd5.

📒 Files selected for processing (6)
  • src/components/ReleaseCard.tsx
  • src/components/ReleaseTimeline.tsx
  • src/store/useAppStore.test.ts
  • src/types/index.ts
  • src/utils/releaseAssets.test.ts
  • src/utils/releaseAssets.ts

Included review availability: Your plan provides up to 8 included reviews per hour; 3 remain after this review.

📝 Walkthrough

Walkthrough

Changes

The release refresh now records added or changed asset IDs. Repository timelines use effective release times that include asset updates. Download rows and repository headers display asset-update indicators for unread releases.

Release asset updates

Layer / File(s) Summary
Asset change detection
src/types/index.ts, src/utils/releaseAssets.ts, src/utils/releaseAssets.test.ts, src/store/useAppStore.test.ts		 Release now stores updated_asset_ids. Asset detection records added and fingerprint-changed assets. Tests cover effective release selection and merged asset IDs.
Effective release timeline
src/components/ReleaseTimeline.tsx		 Repository groups use effective release times for sorting and update status while retaining the latest release for version labels.
Asset update indicators
src/components/ReleaseCard.tsx, src/components/ReleaseTimeline.tsx		 Download metadata includes asset IDs. Unread changed assets and repositories display an “Asset updated” indicator.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to e8a1f

This PR updates release and asset timestamps and adds asset update markers; no actionable merge-blocking risk remains after normal checks and review.

Sequence Diagram(s)

sequenceDiagram
  participant GitHubReleaseAssets
  participant ReleaseRefresh
  participant ReleaseTimeline
  participant ReleaseCard
  GitHubReleaseAssets->>ReleaseRefresh: provide refreshed release assets
  ReleaseRefresh->>ReleaseRefresh: compute updated_asset_ids
  ReleaseRefresh-->>ReleaseTimeline: provide changed releases
  ReleaseTimeline->>ReleaseTimeline: select latest effective release
  ReleaseTimeline-->>ReleaseCard: pass assetId and update metadata
  ReleaseCard->>ReleaseCard: display “Asset updated” when unread
Loading

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the release update time and asset marker fixes.
Linked Issues check ✅ Passed The changes detect added and changed assets and refresh related release update information as required by [#263].
Out of Scope Changes check ✅ Passed The implementation and tests remain focused on release asset detection, update times, and asset update indicators.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/release-asset-update-indicators

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@AmintaCCCP
AmintaCCCP merged commit 02785b2 into main Aug 19, 2026
5 checks passed
@AmintaCCCP
AmintaCCCP deleted the fix/release-asset-update-indicators branch August 19, 2026 15:20
This was referenced Aug 20, 2026
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug] release里提取不全

1 participant

@AmintaCCCP